Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new aerocom namelist #35

Merged

Conversation

mvertens
Copy link
Contributor

  • remove AEROCOM as a cpp variable - and introduce a new namelist to turn it on
  • add a new file oslo_aero_diagnostics.F90 that activates oslo_aero (and aercom namelists) and is called from cam_diagnostics.F90. This makes it much easier to modularize where the oslo-aero diagnostics fields are defined.
  • verified that running SMS_Ln9.ne30pg3_ne30pg3_mtn14.NF1850.derecho_intel.cam-outfrq9s with use_aerocom=.false. have bfb the same history files
  • verified that running SMS_Ln9.ne30pg3_ne30pg3_mtn14.NF1850.derecho_intel.cam-outfrq9s with use_aerocom=.true. added the aerocom variables.
  • a question about the new aerocom output appears in Diagnostic changes with AEROCOM noresm3_dev_simulations#33
  • NOTE: this PR must also be accompanied my PRs to https://github.com/NorESMhub/CAM and to https://github.com/NorESMhub/CAM-Nor-physics

@mvertens mvertens requested a review from gold2718 August 28, 2024 15:05
@mvertens mvertens added the enhancement New feature or request label Aug 28, 2024
@mvertens mvertens added this to the NorESM2.5 milestone Aug 28, 2024
@gold2718 gold2718 force-pushed the feature/new_aerocom_namelist branch from a35a70b to 0db5173 Compare August 28, 2024 15:26
Add OSLO Aero diagnostics to physpkg
Copy link
Contributor

@gold2718 gold2718 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Namelist variables should be protected.
Several files have long lines that need to be trimmed (#36).

! Namelist variables:
real(r8) :: volc_fraction_coarse = 0.0_r8 !Fraction of volcanic aerosols in coarse mode
! Public Namelist variables:
logical, public :: use_aerocom = .false. ! If true, turn on aerocom output
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be protected

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fields with unitless units should be updated to something descriptive (see #37).

Copy link
Contributor

@gold2718 gold2718 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now.

@gold2718 gold2718 merged commit 87f76b3 into NorESMhub:noresm_develop Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants